Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added in srst2 recipes for 0.1.4.6 and 0.2.0 #3767

Merged
merged 1 commit into from
Feb 13, 2017
Merged

Added in srst2 recipes for 0.1.4.6 and 0.2.0 #3767

merged 1 commit into from
Feb 13, 2017

Conversation

ericenns
Copy link
Contributor

@ericenns ericenns commented Feb 10, 2017

  • I have read the guidelines above.
  • This PR adds a new recipe.
  • This PR updates an existing recipe.
  • This PR does something else (explain below).

- numpy >=1.7.1
- scipy >=0.12.0
- bowtie2 >=2.1.0
- samtools ==0.1.18
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you need setuptools in your run section?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

setuptools is just needed to be able to install the python package, from looking at other recipes it only appears to be included in the build section.

- numpy >=1.7.1
- scipy >=0.12.0
- bowtie2 >=2.1.0
- samtools ==0.1.18
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe this can be removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure what your askin. If it is in regards to the 0.1.4.6 recipe, I do need it as we have a galaxy tool for 0.1.4.6 that we would like to continue using until we have time to validate 0.2 for our purposes.

@ericenns ericenns merged commit fe7ba60 into master Feb 13, 2017
@ericenns ericenns deleted the add-srst2 branch February 13, 2017 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants